The code looks logical to me as in calling the post_login method before redirecting.. I took some help from QE to test it and am not sure if the issue is reproducible though.

I'm not sure if I completely understand the issue here, but looking at the fix it seems @hao-yu is just moving the user.post_successful_login from extlogin to login_user(user). So is the issue we are trying to solve here the Last login time not being updated when we log in via WebUI using the AD credentials? Because when I log in using kinit the Last login time gets updated just ok (~ I could not reproduce the issue).

Also I wonder if we need to keep user.post_successful_login at both places to get the time updated no matter how we log in (via kinit or WebUI) or if keeping it only in login_user(user) works just fine for both?

I'm sorry for my ignorance, not an expert here, I just got somehow curious about this one.

Thank you!